Skip to content

Conversation

@leebs0521
Copy link
Collaborator

@leebs0521 leebs0521 commented Jan 14, 2025

resolved:

📌 과제 설명

모집글 리팩토링
모집글 스케쥴링 기능

👩‍💻 요구 사항과 구현 내용

  • 모집글 생성 로직 리팩토링
    • RecruitStatus 생성자 추가
    • RecruitBoardCreateRequestDto 에서 RECRUTING 상태로 생성하도록 변경
  • 모집글 수정 로직 리팩토링
  • 모집글 수정시 시간을 기준으로 수정 가능여부 검증
    • 봉사 시작 일시 전날까지만 변경 가능하도록
  • 모집글 수정시, 봉사 시작/종료 일시(변경이 없더라도)는 생성일 기준으로 검증하도록 수정
  • 00시 기준 해당일에 봉사가 시작하는 게시글 모집완료 상태 변경
  • 00시 기준 전날 끝난 게시글 완료 상태 변경

✅ PR 포인트 & 궁금한 점

@leebs0521 leebs0521 self-assigned this Jan 14, 2025
@leebs0521 leebs0521 linked an issue Jan 14, 2025 that may be closed by this pull request
4 tasks
Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다.

디스코드 소통 내용만 봐도 상당히 검증 작업이 많이 필요해보였는데, 깔끔하게 구현해두셔서 편히 읽었습니다~

 - RecruitStatus 기본 값 제거, 생성자 파라미터 추가
 - 검증 로직 제거, 비지니스 로직 추가
 - RecruitStatus 기본 값 제거, 생성자 파라미터 추가
 - 검증 로직 제거, 비지니스 로직 추가
 - 봉사 시작/종료 일시 수정시, 생성일 하루 이후부터 가능하도록 검증
 - 봉사 시작/종료 일시 수정시, 생성일 하루 이후부터 가능하도록 검증
 - 모집글 수정시 시간에 따른 검증 일괄 추가
   - 봉사 시작 일시 전날까지 수정 가능하도록 검증
 - 모집글 수정시 시간에 따른 검증 일괄 추가
   - 봉사 시작 일시 전날까지 수정 가능하도록 검증
 - 봉사 시작일 기준 모집완료(CLOSED) 변경
 - 봉사 종료일 기준 완료(COMPLETED) 변경
 - 봉사 시작일 기준 모집완료(CLOSED) 변경
 - 봉사 종료일 기준 완료(COMPLETED) 변경
 - 최대 시도, 주기 명시적으로 변경
@leebs0521 leebs0521 force-pushed the feature/198-add-recruit-board-schedule branch from e3c6562 to 97c9e20 Compare January 15, 2025 03:38
Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!👍🏻

@sonarqubecloud
Copy link

@leebs0521 leebs0521 merged commit 9325da1 into main Jan 15, 2025
3 checks passed
@leebs0521 leebs0521 deleted the feature/198-add-recruit-board-schedule branch January 15, 2025 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 모집글 스케쥴링

5 participants